Skip to content

Referral Tracking API added#237

Merged
vijaykhollam merged 1 commit into
tekdi:aspire-leadersfrom
Tusharmahajan12:new_aspm
May 21, 2026
Merged

Referral Tracking API added#237
vijaykhollam merged 1 commit into
tekdi:aspire-leadersfrom
Tusharmahajan12:new_aspm

Conversation

@Tusharmahajan12
Copy link
Copy Markdown
Collaborator

No description provided.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 15, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 4aba2a50-a761-40f9-86fc-1649bec1faf1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
97.6% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces several new referral-related API endpoints to the apiConfig.ts file, covering listing, resolving, importing, and updating referrals, as well as service-specific upload and status routes. Feedback focuses on enhancing security by implementing granular PRIVILEGE_CHECK constraints for each new route and ensuring consistency with existing modules. Additionally, the reviewer suggests including these routes in the academic year validation list where applicable and correcting minor comment formatting for better alignment with the codebase style.

Comment thread src/common/middleware/apiConfig.ts
Comment thread src/common/middleware/apiConfig.ts
Comment thread src/common/middleware/apiConfig.ts
Comment thread src/common/middleware/apiConfig.ts
Comment thread src/common/middleware/apiConfig.ts
Comment thread src/common/middleware/apiConfig.ts
@vijaykhollam vijaykhollam merged commit 13c308f into tekdi:aspire-leaders May 21, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants